Skip to content

Fix Settings dropdown z-index overlap and Activity Logs filter panel - #30

Closed
teja-311 wants to merge 1 commit into
udaycodespace:mainfrom
teja-311:fix/navbar-dropdown-and-filter-overflow
Closed

Fix Settings dropdown z-index overlap and Activity Logs filter panel#30
teja-311 wants to merge 1 commit into
udaycodespace:mainfrom
teja-311:fix/navbar-dropdown-and-filter-overflow

Conversation

@teja-311

Copy link
Copy Markdown
Contributor

Description

Fixes two layout bugs on the admin dashboard pages:

1. Settings dropdown rendering behind page content
The dropdown (Dark Mode / Toggle Brightness) rendered behind the tab bar (Users/Payments/Activity Logs) instead of on top of it. Fixed by adding position: relative and zIndex: 1040 to the <Navbar> in NavBar.jsx, giving it its own stacking context so the dropdown (zIndex: 2000) reliably renders above page content.

2. Activity Logs filter panel overflowing its container
The filter row (Search/Role/Activity/From/To/Order/Rows) overflowed past the card's right edge due to a fixed 7-column grid in ActivityLogs.css that couldn't shrink below its minimum widths. Changed grid-template-columns to repeat(auto-fit, minmax(140px, 1fr)) so filter boxes wrap onto new rows instead of overflowing.

Closes #24

Before / After

Settings dropdown overlap (before → after):
Screenshot 2026-07-20 181633
Screenshot 2026-07-22 203540

Filter panel overflow (before → after):
Screenshot 2026-07-20 181909
Screenshot 2026-07-22 203600

Steps to Test

  1. Log in as Admin (learn@learnhub.com / changethispassword)
  2. Go to any admin tab (Users, Payments, or Activity Logs) and click Settings in the navbar
  3. Confirm the dropdown renders cleanly above the tab bar, not behind it
  4. Go to the Activity Logs tab
  5. Confirm the filter row (Search/Role/Activity/From/To/Order/Rows) stays within the card's border and wraps onto a new row instead of overflowing

Build Verification

Ran npm run dev in frontend and it builds and runs with no new warnings or errors.

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)

Checklist

  • My code follows the style guidelines of this project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings or console errors.
  • I have tested my changes locally and verified they work as expected.
  • Any dependent changes have been merged and published in downstream modules.

…overflow

Signed-off-by: Tejashwini <deviteja76@gmail.com>
@udaycodespace udaycodespace added redo Reviewed — needs changes before it can be merged and removed frontend labels Jul 22, 2026
@udaycodespace

Copy link
Copy Markdown
Owner

Thanks for the PR! The fixes are valid, but this only contains a couple of small CSS changes.

  1. As mentioned before, I'm trying to avoid contribution farming, so I'll be closing this PR.
  2. Please look for a more meaningful issue related to the codebase.
  3. If you can't find one, let me know and I'll help you pick one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

redo Reviewed — needs changes before it can be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Settings dropdown overlap and Activity Logs filter panel overflow on admin pages

2 participants